Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Rename executiondate to occurreddate in DB [TECH-1615] #15648

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

enricocolasante
Copy link
Contributor

Following on #15642

This PR aims to replace all occurrences of executionDate in the DB and repository layer to occurredDate.
The general rule that we are using is that date fields are suffixed with At in the API and then suffixed with Date in the service and the DB layer.

@enricocolasante enricocolasante added the run-api-analytics-tests Enables analytics e2e tests label Nov 10, 2023
…hisp/dhis/analytics/common/params/dimension/DimensionParam.java
Copy link

sonarcloud bot commented Nov 10, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
3.6% 3.6% Duplication

Copy link

codecov bot commented Nov 10, 2023

Codecov Report

Merging #15648 (045c3af) into master (21d1572) will decrease coverage by 0.01%.
The diff coverage is 59.09%.

@@             Coverage Diff              @@
##             master   #15648      +/-   ##
============================================
- Coverage     66.21%   66.21%   -0.01%     
+ Complexity    31267    31265       -2     
============================================
  Files          3487     3487              
  Lines        129904   129904              
  Branches      15170    15170              
============================================
- Hits          86013    86010       -3     
- Misses        36812    36813       +1     
- Partials       7079     7081       +2     
Flag Coverage Δ
integration 49.77% <54.54%> (-0.01%) ⬇️
integration-h2 32.40% <13.63%> (-0.01%) ⬇️
unit 30.32% <22.72%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...c/main/java/org/hisp/dhis/analytics/TimeField.java 91.30% <100.00%> (ø)
...org/hisp/dhis/program/AnalyticsPeriodBoundary.java 74.57% <ø> (-3.39%) ⬇️
...ytics/event/data/DefaultEventDataQueryService.java 83.26% <ø> (ø)
...ics/event/data/EnrollmentTimeFieldSqlRenderer.java 32.75% <ø> (ø)
...ics/event/data/JdbcEnrollmentAnalyticsManager.java 51.65% <100.00%> (ø)
...nalytics/event/data/JdbcEventAnalyticsManager.java 65.94% <ø> (ø)
...nalytics/table/JdbcEventAnalyticsTableManager.java 81.66% <100.00%> (ø)
...tics/table/JdbcTeiEventsAnalyticsTableManager.java 86.60% <100.00%> (ø)
...s/tei/query/context/querybuilder/ContextUtils.java 0.00% <ø> (ø)
...ext/querybuilder/ProgramIndicatorQueryBuilder.java 1.01% <ø> (ø)
... and 17 more

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 21d1572...045c3af. Read the comment docs.

Copy link
Contributor

@gnespolino gnespolino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, api-e2e runs fine, so it should be good to go

@enricocolasante enricocolasante merged commit 1d0602a into master Nov 14, 2023
18 checks passed
@enricocolasante enricocolasante deleted the TECH-1615 branch November 14, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-api-analytics-tests Enables analytics e2e tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants